From 80bc9d773cc550e9ca448741ac174b54c61073b6 Mon Sep 17 00:00:00 2001
From: Carlos Garnacho <carlosg@gnome.org>
Date: Fri, 26 Sep 2025 05:25:15 +0200
Subject: [PATCH] panelMenu: Port to using ClickGesture

Only enable this gesture if a menu is set, so there's no
overlap with other gestures in buttons without a menu (e.g.
the ActivitiesButton).

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3912>
---
 js/ui/panelMenu.js | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/js/ui/panelMenu.js b/js/ui/panelMenu.js
index a46a78b2b..b2ec479e5 100644
--- a/js/ui/panelMenu.js
+++ b/js/ui/panelMenu.js
@@ -102,6 +102,14 @@ export const Button = GObject.registerClass({
             accessible_role: Atk.Role.MENU,
         });
 
+        this._clickGesture = new Clutter.ClickGesture();
+        this._clickGesture.set_recognize_on_press(true);
+        this._clickGesture.connect('recognize', () => {
+            this.menu?.toggle();
+        });
+        this._clickGesture.set_enabled(!dontCreateMenu);
+        this.add_action(this._clickGesture);
+
         if (dontCreateMenu)
             this.menu = new PopupMenu.PopupDummyMenu(this);
         else
@@ -130,18 +138,10 @@ export const Button = GObject.registerClass({
             Main.uiGroup.add_child(this.menu.actor);
             this.menu.actor.hide();
         }
+        this._clickGesture.set_enabled(this.menu !== null);
         this.emit('menu-set');
     }
 
-    vfunc_event(event) {
-        if (this.menu &&
-            (event.type() === Clutter.EventType.TOUCH_BEGIN ||
-             event.type() === Clutter.EventType.BUTTON_PRESS))
-            this.menu.toggle();
-
-        return Clutter.EVENT_PROPAGATE;
-    }
-
     vfunc_hide() {
         super.vfunc_hide();
 
-- 
2.39.5